Skip to content

Feat/verify protected routes#30

Merged
AAdewunmi merged 5 commits into
mainfrom
feat/verify-protected-routes
May 7, 2026
Merged

Feat/verify protected routes#30
AAdewunmi merged 5 commits into
mainfrom
feat/verify-protected-routes

Conversation

@AAdewunmi

Copy link
Copy Markdown
Owner

Summary

Verify the protected dashboard route, role-aware permission helpers, and Sprint 1 integration baseline.

Changes

  • Added Friday console verification runbook for protected dashboard and role access checks.
  • Verified dashboard route registration, URL resolution, and template discovery.
  • Verified anonymous users are redirected away from the dashboard.
  • Verified authenticated users can access the dashboard and land there after login.
  • Verified authenticated and public navigation states.
  • Verified role permission helpers for anonymous users, superusers, users without roles, and users with roles.
  • Verified role_required() blocks missing roles and allows matching roles.
  • Updated verification expectations to the current 48 test baseline.

Behavior

  • /dashboard/ remains protected by authentication.
  • Login redirects authenticated users to the dashboard.
  • Role display works through view context.
  • Permission decisions remain centralized in Python helpers/decorators.
  • Verification data uses isolated Friday-only users and role slugs.

Why

This confirms Sprint 1 has a reliable protected UI path and a tested role-permission foundation before moving into Sprint 2 study-session data.

Validation

  • Friday runbook passed end to end.
  • Dashboard tests passed.
  • Role and permission tests passed.
  • Top-level integration tests passed.
  • Full isolated suite passed with 48 passed.
  • Black, Ruff, and isort passed.
  • Migrations are clean.

Result

Sprint 1 protected routes, dashboard access, and role helper behavior are verified and ready for completion review.

Notes

  • The dashboard is authentication-protected, not role-protected.
  • The verification role uses friday-student to avoid touching common/local role data.
  • Role checks use user.studybuddy_roles, matching the current role model.
  • Templates only display role context; permission decisions stay in helpers/decorators.
  • The current expected suite baseline is 48 tests.
  • The Day 5 runbook is executable at docs/sprint-runbook/sprint-1/sprint-1-day-5.sh.

@codecov

codecov Bot commented May 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@AAdewunmi AAdewunmi merged commit d7bef13 into main May 7, 2026
2 checks passed
@AAdewunmi AAdewunmi deleted the feat/verify-protected-routes branch May 7, 2026 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

Verify protected routes

1 participant